Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to numba>=0.57. #648

Merged
merged 1 commit into from
May 26, 2023
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented May 26, 2023

cuDF now requires numba>=0.57.

I am not sure if this change is strictly needed or if cuDF's requirements will automatically enforce the greater lower bound.

@bdice bdice requested a review from a team as a code owner May 26, 2023 15:16
Copy link
Contributor

@gmarkall gmarkall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I don't know so much about the use of this repo / tooling, but I note it follows the same pattern as the merged #535 and I think generally ensuring the Numba version is 0.57 is 👍

@raydouglass raydouglass merged commit 0add688 into rapidsai:branch-23.06 May 26, 2023
@gmarkall gmarkall mentioned this pull request Aug 2, 2024
copy-pr-bot bot pushed a commit that referenced this pull request Aug 2, 2024
numba-cuda is the NVIDIA-maintained CUDA target for Numba, which depends
on the numba package.

This PR replaces the numba dependency with numba-cuda. However, similar to
@bdice's comment in #648, I don't know if we strictly need this here any
more due to the cudf dependency (from rapidsai/cudf#16474) ensuring
numba-cuda is present.
rapids-bot bot pushed a commit that referenced this pull request Oct 22, 2024
**EDIT:** This now removes the numba dependency, which comes transitively from cuDF.

**Original description:**

numba-cuda is the NVIDIA-maintained CUDA target for Numba, which depends on the numba package.

This PR replaces the numba dependency with numba-cuda. However, similar to @bdice's comment in #648, I don't know if we strictly need this here any more due to the cudf dependency (from rapidsai/cudf#16474) ensuring numba-cuda is present.

Authors:
  - Graham Markall (https://github.com/gmarkall)
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #715
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants